Skip to content

[BoundsSafety] Fix Sema/compound-literal-ended_by.c test #10702

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 21, 2025

Conversation

patrykstefanski
Copy link

After d58d5d6, we emit an error earlier for compound literals that occur outside the body of a function and are not constant expressions.

rdar://151424542

@patrykstefanski patrykstefanski added the clang:bounds-safety Issue relating to the experimental -fbounds-safety feature in Clang label May 19, 2025
@patrykstefanski
Copy link
Author

@swift-ci test llvm

@patrykstefanski patrykstefanski force-pushed the eng/pstefanski/PR-151424542 branch from 77591e7 to 0aa1dd2 Compare May 20, 2025 13:40
@patrykstefanski
Copy link
Author

@swift-ci test llvm

After d58d5d6, we emit an error earlier
for compound literals that occur outside the body of a function and are
not constant expressions.

rdar://151424542
@patrykstefanski patrykstefanski force-pushed the eng/pstefanski/PR-151424542 branch from 0aa1dd2 to 1980516 Compare May 20, 2025 13:54
Copy link

@delcypher delcypher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks for fixing this.

@delcypher
Copy link

Going to merge this now to fix CI.

@delcypher delcypher merged commit 5936f49 into next May 21, 2025
@delcypher delcypher deleted the eng/pstefanski/PR-151424542 branch May 21, 2025 00:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clang:bounds-safety Issue relating to the experimental -fbounds-safety feature in Clang
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants